homework-tk by tatiana-kornienko14 · Pull Request #14 · DafeMipt212/homeworks_cpp

tatiana-kornienko14 · 2022-12-12T14:08:04Z

No description provided.

LostPointer · 2022-12-12T18:30:17Z


 std::vector<std::string> SplitString(const std::string& data) {
-  return {};
+  std::vector<std::string> v;


плохое имя у переменной, непонятно что она означает

LostPointer · 2022-12-12T18:31:05Z

+  std::vector<std::string> v;
+  std::string tmp = "";
+
+  for (auto c : s) {


тут ошибка компиляции будет, нет переменной s

LostPointer · 2022-12-12T18:32:31Z

+  std::string tmp = "";
+
+  for (auto c : s) {
+    if (c != ' ') {


если посмотреть в тесты, то там не только по пробелу разделение идет, но и по символу табуляции ('\t')

LostPointer · 2022-12-12T18:39:38Z

-
+#include <regex>
 #include <stack>
+#define vector_string std::vector<std::string>


using vector_string = std::vector<std::string>;

так же можно использовать typedef

не стоит использовать define где этого можно избежать

LostPointer · 2022-12-12T18:41:27Z


 int Calculate(const std::string& data) {
-  return 0;
+    std::vector<int> v1; // хотела сделать тип double, но в тестах int, так что пока так


если хочешь сделай double, это только плюс

название переменной так себе (непонятно что означает)

LostPointer · 2022-12-12T18:43:11Z

-  return 0;
+    std::vector<int> v1; // хотела сделать тип double, но в тестах int, так что пока так
+    std::string tmp;
+    bool f = 0, pr = 0, dl = 0, mns = 0;


совсем непонятные названия переменных, и хорошая практика: одна переменная - одна строка

LostPointer · 2022-12-12T18:47:35Z

+    bool f = 0, pr = 0, dl = 0, mns = 0;
+
+    for (size_t i = 0; i < data.size(); ++i) {
+        if (data[i] != "*" && data[i] != "+" && data[i] != "-" && data[i] != "/" && !(is_numb(data[i]))) {


!(is_numb(data[i])) лишние круглые скобки

а ещё лучше вот так оформить:

if (data[i] != "*" && data[i] != "+" && data[i] != "-" && data[i] != "/" && !is_numb(data[i]))

LostPointer · 2022-12-12T18:49:02Z

+
+    for (size_t i = 0; i < data.size(); ++i) {
+        if (data[i] != "*" && data[i] != "+" && data[i] != "-" && data[i] != "/" && !(is_numb(data[i]))) {
+            std::cout << "ошибка!\n";


тут можно кинуть исключение, например std::runtime_error

LostPointer · 2022-12-12T18:53:36Z

в первой задаче не рассмотрен случай со скобками и табуляцией

ilya-kadochnikov · 2022-12-12T22:11:37Z

 #include <stack>
+#define vector_string std::vector<std::string>
+
+bool is_numb(const std::string &str) {


В большинстве случаев не стоит сокращать слова в именах. Имя IsNumber (ну или is_number) проще понять, чем is_numb (к тому же numb переводится с английского как онемевший, что добавляет двусмысленности)

ilya-kadochnikov · 2022-12-12T22:27:37Z

+    bool f = 0, pr = 0, dl = 0, mns = 0;
+
+    for (size_t i = 0; i < data.size(); ++i) {
+        if (data[i] != "*" && data[i] != "+" && data[i] != "-" && data[i] != "/" && !(is_numb(data[i]))) {


Следующий код не скомпилируется
if (data[i] != "*" && data[i] != "+" && data[i] != "-" && data[i] != "/" && !(is_numb(data[i]))).
Первая причина: data[i] имеет тип char, а символы "*", "+", "-" и "/" тип const char*, и сравнивать их нельзя.
Вторая причина: функция is_numb принимает аргумент типа const std::string &str, а вы передаёте char.
Можно исправить эту строчку так:
if (data[i] != '*' && data[i] != '+' && data[i] != '-' && data[i] != '/' && !(is_numb(std::to_string(data[i]))))

ilya-kadochnikov · 2022-12-12T22:29:17Z

+        }
+
+        if (f) {
+            if (data[i] == ")") {


Не скомпилируется. Исправление: data[i] == ')'

ilya-kadochnikov · 2022-12-12T22:29:53Z

+            continue;
+        }
+
+        if (data[i] == "(") {


data[i] == '('

ilya-kadochnikov · 2022-12-12T22:36:22Z

+        }
+
+        if (pr) {
+            v1[v1.size()-1] *= std::stoi(data[i]);


Не скомпилируется. Функция std::stoi принимает строку, не char. Вариант исправления:

v1[v1.size() - 1] *= data[i] - '0';

ilya-kadochnikov · 2022-12-12T22:41:00Z

+            continue;
+        }
+
+        if (data[i] == "*") {


data[i] == '*'

ilya-kadochnikov · 2022-12-12T22:41:17Z

+            continue;
+        }
+
+        if (data[i] == "/") {


data[i] == '/'

ilya-kadochnikov · 2022-12-12T22:41:47Z

+            continue;
+        }
+
+        if (data[i] == "-") {


data[i] == '-'

ilya-kadochnikov · 2022-12-12T22:42:13Z

+            continue;
+        }
+
+        if (data[i] == "+") {


data[i] == '+'

ilya-kadochnikov · 2022-12-12T22:43:56Z

+            continue;
+        }
+
+        v1.push_back(std::stoi(data[i]));


v1.push_back(data[i] - '0');

поправила названия переменных, добавила проверку на табуляцию

я так посмотрела, define у меня в принципе нигде не используется... упс увидела, кстати, у себя заметную проблему в коде, скоро исправлю, заодно доисправляю названия переменных

LostPointer

поправь в тестах для первого задания с { "a", "(a a)", "b", "(asd as)" } на { "a", "a a", "b", "asd as" }, так будут проходить тесты

LostPointer · 2022-12-15T07:02:55Z

+        is_bracket = 0;
+        if (tmp != "") {
+          answer.push_back(tmp);
+          tmp = "";


лучше оспользовать метод clear: tmp.clear()
это даст немного больше понимания что ты очищаешь переменную, а не присваиваешь значение переменной

LostPointer · 2022-12-15T07:05:54Z

+#include <iostream>
+#include <regex>
+#include <stack>
 #include "utils.hpp"


а вот если у тебя два файла с одинаковым мененм и расширениями .cpp и .hpp, в cpp инклуд hpp должен идти первым. получится примерно вот так:

#include "utils.hpp" #include <regex> #include <stack> #include <iostream>

LostPointer · 2022-12-15T07:08:54Z

+        }
+
+        if (!is_number(number)) {
+            throw ("ошибка!\n");


LostPointer · 2022-12-15T07:09:24Z

+        }
+
+        if (multipl) {
+            terms[terms.size()-1] *= std::stoi(number);


пробелы пропущены: terms.size() - 1

ilya-kadochnikov · 2022-12-15T22:32:42Z

+            number += data[i];
+            continue;
+        }
+


Скобочки у вас пока ещё корректно не обрабатываются. Наличие скобочек приводит к вылетам программы. Чтобы исправить вылеты, вставьте сюда код (но это ещё не всё):

if (number.empty()) continue;

ilya-kadochnikov · 2022-12-15T22:36:13Z

+        minus = 0;
+    } else {
+        terms.push_back(std::stoi(number));
+    }


Чтобы исправить вылеты из-за скобочек, окружите операторы if-else ещё одной проверкой:

if (!number.empty()) { if (multipl) { terms[terms.size() - 1] *= std::stoi(number); multipl = 0; } else if (division) { terms[terms.size() - 1] /= std::stoi(number); division = 0; } else if (minus) { terms.push_back(-std::stoi(number)); minus = 0; } else { terms.push_back(std::stoi(number)); } }

ilya-kadochnikov · 2022-12-15T22:42:08Z

+
+    for (size_t i = 0; i < data.size(); ++i) {
+        if (is_brackets) {
+            if (data[i] == ')') {


Замечу просто, что если отключать флаг is_brackets сразу, как только встречается закрывающая скобка, то вложенные друг в друга скобки точно корректно оброватываться не будут. Впрочем, для первого задания и так хорошо

ilya-kadochnikov · 2022-12-15T22:49:50Z

+        }
+
+        if (multipl) {
+            terms[terms.size() - 1] *= std::stoi(number);


std::stoi(number) возвращает int, а term хранит double (да и функция Calculate, вроде, должна double возвращать). Нестыковка)

дз Корниенко

5b48ec8

tatiana-kornienko14 changed the title ~~дз Корниенко~~ homework-tk Dec 12, 2022

tatiana-kornienko14 added 2 commits December 12, 2022 18:06

Update utils.cpp

927d13d

Update utils.cpp

d7e0ebf

LostPointer requested changes Dec 12, 2022

View reviewed changes

ilya-kadochnikov reviewed Dec 12, 2022

View reviewed changes

tatiana-kornienko14 and others added 11 commits December 13, 2022 05:32

Update utils.cpp

967dcd6

поправила названия переменных, добавила проверку на табуляцию

поменяла названияя переменных, скоро поправлю еще кое-что

4478790

почти доделала, осталось немного

583d79f

вроде все поправила

cbf9a23

доделала; вот это уже должно работать

cbe78b7

обработала ошибку в main.cpp

9d256ec

это не особо важно

eef1f60

добавила условие со скобками, надо еще доработать

e10e177

вроде добила

ad1ddc4

пара поправок

a3f08ef

tatiana-kornienko14 requested a review from LostPointer December 14, 2022 18:23

LostPointer requested changes Dec 15, 2022

View reviewed changes

Tatyana145 added 2 commits December 15, 2022 14:22

поправила

cffd546

поправила

82eaff7

ilya-kadochnikov reviewed Dec 15, 2022

View reviewed changes

Tatyana145 added 4 commits December 16, 2022 20:50

пока временно поменяла все на int, потом верну обратно; добавила empty

b9d468e

СКОБОЧКИ РАБОТАЮТ, УРА :D а вообще осталось чуть-чуть

f95057c

теперь скобочки работают и внутри скобочек

6165635

работает! :D

7af5548

Uh oh!

Conversation

tatiana-kornienko14 commented Dec 12, 2022

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

LostPointer commented Dec 12, 2022

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

LostPointer left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants